Skip to content

Conversation

@ryan953
Copy link
Member

@ryan953 ryan953 commented Oct 8, 2024

This creates a new command yarn https-proxy to help people use their dev servers over https.

The json file content was generated using caddy adapt with this config as input:

:8003
reverse_proxy localhost:8000
tls ./config/localhost.pem ./config/localhost-key.pem

Relates to getsentry/sentry-docs#11520

@ryan953 ryan953 requested a review from a team as a code owner October 8, 2024 16:51
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 8, 2024
@ryan953 ryan953 requested a review from joshuarli October 8, 2024 16:55
@scttcper
Copy link
Member

scttcper commented Oct 8, 2024

I think i'm confused what this is for, i thought everyone was using https locally via webpack dev server

@ryan953
Copy link
Member Author

ryan953 commented Oct 8, 2024

I think i'm confused what this is for, i thought everyone was using https locally via webpack dev server

I was trying to use sentry devserver with https, and had all the self-signed cert stuff setup but using https wasn't working. It's possible that I could've messed around with the config files a lot more, but idk if the the python server has paths to terminate TLS, and to find the self-signed cert and key files. I don't even want to look and dig into that, I prefer a reverse proxy whether it's in the docs or not :)

The original docs evan wrote said "and it'll just work" but that wasn't true for me.


When i use yarn dev-ui, looking now i see that it's using https at port 7999 so that's working well.

@ryan953 ryan953 merged commit 347732a into master Oct 8, 2024
41 checks passed
@ryan953 ryan953 deleted the ryan953/https-proxy branch October 8, 2024 20:18
@github-actions github-actions bot locked and limited conversation to collaborators Oct 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants